Skip to content

Deduplicate join output under union with aggregation#11935

Merged
sopel39 merged 1 commit intotrinodb:masterfrom
gaurav8297:gaurav8297/union_dedup_join
Apr 25, 2022
Merged

Deduplicate join output under union with aggregation#11935
sopel39 merged 1 commit intotrinodb:masterfrom
gaurav8297:gaurav8297/union_dedup_join

Conversation

@gaurav8297
Copy link
Copy Markdown
Member

@gaurav8297 gaurav8297 commented Apr 13, 2022

Description

Is this change a fix, improvement, new feature, refactoring, or other?

improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

core query engine

How would you describe this change to a non-technical end user or system administrator?

Related issues, pull requests, and links

Documentation

( ) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@raunaqmorarka raunaqmorarka requested a review from sopel39 April 13, 2022 16:08
@raunaqmorarka
Copy link
Copy Markdown
Member

@gaurav8297 can you take care of test failures here ?

@gaurav8297
Copy link
Copy Markdown
Member Author

@gaurav8297 can you take care of test failures here ?

Sure

@raunaqmorarka
Copy link
Copy Markdown
Member

@gaurav8297 can you also post tpcds results for q75 partitioned and unpartitioned ?

@gaurav8297
Copy link
Copy Markdown
Member Author

gaurav8297 commented Apr 22, 2022

union_dedup_benchmark.pdf

Tpcds q75 benchmarks cc @raunaqmorarka

@gaurav8297
Copy link
Copy Markdown
Member Author

@sopel39 PTAL

@sopel39
Copy link
Copy Markdown
Member

sopel39 commented Apr 25, 2022

Failed due to #11848

@sopel39 sopel39 merged commit 9352ebc into trinodb:master Apr 25, 2022
@sopel39 sopel39 mentioned this pull request Apr 25, 2022
@github-actions github-actions bot added this to the 379 milestone Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants